chore: sitemap paths added into the robots.txt#8820
chore: sitemap paths added into the robots.txt#8820canerakdas wants to merge 1 commit intonodejs:mainfrom
Conversation
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 1f801d9. Bugbot is set up for automated code reviews on this repo. Configure here. |
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website Please review the changes when you have a chance. Thank you! 🙏 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8820 +/- ##
==========================================
- Coverage 74.27% 74.25% -0.03%
==========================================
Files 104 104
Lines 8847 8847
Branches 328 328
==========================================
- Hits 6571 6569 -2
- Misses 2274 2276 +2
Partials 2 2 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Pull request overview
Adds sitemap references to the dynamically generated robots.txt so crawlers can discover multiple sitemap files.
Changes:
- Import
BASE_URLfor constructing absolute sitemap URLs. - Add two sitemap entries (
/sitemap.xmland/learn/sitemap.xml) to theMetadataRoute.Robotsoutput.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
Since we now have multiple sitemap files, the sitemap paths have been added to robots.txt for the SEO.
Validation
In preview environment the robots.txt should contains
Related Issues
Related with this comment; nodejs/learn#47 (review)
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.